Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: fund test suite's account by default in local-test node #5935

Merged
merged 3 commits into from
May 23, 2024

Conversation

kayagokalp
Copy link
Member

Description

Funds account associated with de97d8624a438121b86a1956544bd72ed68cd69f2c99555b08b1e8c51ffd511c which is used in the test suite. This is done since locally we tend to rely on ci's initial state which should include this account so that some deployment transactions (required for updating contract ids) can be executed.

@kayagokalp kayagokalp added the testing General testing label Apr 30, 2024
@kayagokalp kayagokalp self-assigned this Apr 30, 2024
@kayagokalp kayagokalp requested a review from a team April 30, 2024 05:17
@kayagokalp kayagokalp enabled auto-merge (squash) April 30, 2024 05:18
@JoshuaBatty JoshuaBatty requested a review from a team April 30, 2024 05:20
@sdankel
Copy link
Member

sdankel commented May 10, 2024

Deployment failed due to insufficient funds. Please be sure to have enough coins to pay for deployment transaction.

Do we need to add more to the account?

@kayagokalp kayagokalp merged commit b51bdb2 into master May 23, 2024
34 of 35 checks passed
@kayagokalp kayagokalp deleted the kayagokalp/fund-test-account branch May 23, 2024 15:40
kayagokalp added a commit that referenced this pull request May 24, 2024
## Description

Fixes the implementation at #5935 and removes bunch of unnecessary
accounts for the CI from state_config. Basically funds the account used
by default to sign via `forc-deploy --default-signer` and the test
suite.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing General testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants